Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chart editor - Set playspeed after variation load #4212

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Lasercar
Copy link
Contributor

Does this PR close any issues? If so, link them below.

Fixes #3041

Briefly describe the issue(s) fixed.

When a variation is loaded (or an audiotrack is reloaded), the new audiotracks don't have their playback speed set.

This PR fixes that, by doing after they're loaded.

Include any relevant screenshots or videos.

@Lasercar Lasercar changed the base branch from main to develop February 23, 2025 03:58
@github-actions github-actions bot added size: small A small pull request with 10 or fewer changes. pr: haxe PR modifies game code. status: pending triage Awaiting review. and removed size: small A small pull request with 10 or fewer changes. labels Feb 23, 2025
@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. topic: chart editor Related to the operation of the Chart Editor. size: small A small pull request with 10 or fewer changes. labels Feb 23, 2025
Copy link
Collaborator

@Hundrec Hundrec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't work in my testing, not sure why!

@Hundrec Hundrec added status: needs revision Cannot be approved because it is awaiting some work by the contributor. and removed status: pending triage Awaiting review. labels Feb 27, 2025
@Lasercar Lasercar force-pushed the chart-editor-set-play-speed-after-variation-load branch from ae43086 to 8e8e7f1 Compare February 28, 2025 11:51
@Lasercar
Copy link
Contributor Author

This doesn't work in my testing, not sure why!

Whoops, that's because I was dumb and didn't notice that I put the pitch changes in the audioInstTrack.onComplete function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. status: needs revision Cannot be approved because it is awaiting some work by the contributor. topic: chart editor Related to the operation of the Chart Editor. type: minor bug Involves a minor bug or issue.
Projects
None yet
3 participants